Skip to content

docs(skill): add on-invocation guard to socratic-code-theory-recovery#480

Merged
rdmueller merged 1 commit into
LLM-Coding:mainfrom
raifdmueller:feat/skill-on-invocation-guard
May 14, 2026
Merged

docs(skill): add on-invocation guard to socratic-code-theory-recovery#480
rdmueller merged 1 commit into
LLM-Coding:mainfrom
raifdmueller:feat/skill-on-invocation-guard

Conversation

@raifdmueller
Copy link
Copy Markdown
Contributor

Follow-up to #478 / #479.

Problem

When a user invokes `/semantic-anchors:socratic-code-theory-recovery`, the LLM loads the SKILL.md content but has no clear instruction on what to do next. Symptom seen in practice: the slash command's output is displayed, and the model defaults to a meta-conversation about the skill ("here's what it does, give it a try sometime") instead of actually starting Phase 1.

Fix

Prepend an "On invocation" section to `SKILL.md` with four explicit steps:

  1. Check whether the same message that invoked the skill (or the immediately preceding context) names a bounded-context path + name.
  2. If not, ask a single specific question for it. Do not default to the current working directory — Phase 1 produces files and a wrong target wastes work.
  3. Run Phase 1 with the user's path (using the existing `prompts/phase-1-question-tree.md` prompt).
  4. Stop after Phase 1. Phase 2 needs team answers; do not run it silently in the same session.

This converts the skill from a passive document into an active behaviour spec without changing the methodology or any of the references.

Affected files

  • `skill/socratic-code-theory-recovery/SKILL.md` — canonical source
  • `plugins/semantic-anchors/skills/socratic-code-theory-recovery/SKILL.md` — auto-mirrored by the pre-commit hook

🤖 Generated with Claude Code

The skill described the methodology but did not tell the LLM what to do
the moment it is invoked. Result: invoking the slash command displayed
the SKILL.md content, and Claude defaulted to a meta-conversation about
the skill instead of starting Phase 1. The user has to know to either
add the bounded-context path to the invoking message or chase it with a
follow-up.

Add a four-step "On invocation" section at the top of SKILL.md:

1. Check whether the user already named a bounded context (path + name).
2. If not, ask one specific question for it — do not assume the current
   working directory, because Phase 1 produces files and a wrong target
   wastes work.
3. Run Phase 1 with the user's path.
4. Stop after Phase 1; Phase 2 needs team answers and must not run
   silently in the same session.

The pre-commit hook mirrors this into the plugin bundle.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 14, 2026

Warning

Rate limit exceeded

@raifdmueller has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 39 minutes and 14 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 257ba7d1-768c-47cb-beec-ea3be1fb734b

📥 Commits

Reviewing files that changed from the base of the PR and between d9190c2 and b0a2ebd.

📒 Files selected for processing (2)
  • plugins/semantic-anchors/skills/socratic-code-theory-recovery/SKILL.md
  • skill/socratic-code-theory-recovery/SKILL.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rdmueller rdmueller merged commit 598d1b6 into LLM-Coding:main May 14, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants